Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document server version, variant, & constant changes #149

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TheRealPear
Copy link
Contributor

@TheRealPear TheRealPear commented Sep 23, 2024

Comment on lines 163 to 164
| `min-server-version` | The minimum server version this variant can run on.<br />**Note:** PGM will attempt to detect the map version from its `level.dat` file. |
| `max-server-version` | The maximum server version this variant can run on. |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

referencing variants in if min-server-version makes little sense
also the note here makes no sense

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

referencing variants in if min-server-version makes little sense

These also show up in Map attributes table as well. Are there no map variants that would be using a different version than the original map?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The thing is simply that if is deciding if this block of xml applies or not based on the version. Variant is irrelevant

<if min-server-version="1.9">
  <item>shield</item>
</if>

this is purely a version dependant line of xml, that has NO relation to variants whatsoever. The map can play in 1.8 and shields won't be referenced, or if it plays in 1.9+ then shields are mentioned (imagine this being an itemremove or whatever)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants